Skip to content

feat: implement batch checking for live events in Events API#788

Merged
AndresMorelos merged 1 commit intomasterfrom
fix/re-discovery-fix-live-property
Jan 29, 2026
Merged

feat: implement batch checking for live events in Events API#788
AndresMorelos merged 1 commit intomasterfrom
fix/re-discovery-fix-live-property

Conversation

@AndresMorelos
Copy link
Contributor

  • Added a new method checkLiveEventsForDestinations to allow batch checking of live events for multiple destinations.
  • Updated the API to use a POST request with a JSON body for improved performance and clarity.
  • Enhanced caching mechanism to store live event status per destination with a 5-minute TTL.
  • Modified tests to cover new functionality and ensure accurate live event status retrieval for multiple destinations.

- Added a new method `checkLiveEventsForDestinations` to allow batch checking of live events for multiple destinations.
- Updated the API to use a POST request with a JSON body for improved performance and clarity.
- Enhanced caching mechanism to store live event status per destination with a 5-minute TTL.
- Modified tests to cover new functionality and ensure accurate live event status retrieval for multiple destinations.
@github-actions
Copy link

Pull Request Test Coverage Report for Build 21465690292

Details

  • 78 of 78 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 91.436%

Totals Coverage Status
Change from base Build 21450851878: 0.005%
Covered Lines: 14815
Relevant Lines: 15956

💛 - Coveralls

Copy link
Contributor

@kevinszuchet kevinszuchet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AndresMorelos AndresMorelos merged commit b2ba8c8 into master Jan 29, 2026
2 checks passed
@AndresMorelos AndresMorelos deleted the fix/re-discovery-fix-live-property branch January 29, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants